Summary

This PR fixes JSON unmarshaling for primitive input types in tool execution wrappers. Previously, NewTool created a correctly typed inputType via ensureInputStructType[T](), but the execution wrapper always unmarshaled into T directly — causing failures when T is a primitive like string, int, bool, or float64.

Unmarshaling now uses reflection to construct the appropriate container, ensuring primitives are handled correctly alongside struct types.

The changes affect connect/tools and corresponding tests.

Changes

1. Tool execution wrapper fix

wrapper := func(input string) (string, error) {
        // Previously: always unmarshaled into T directly
        // Now: uses reflection to build the correct container
		ptr := reflect.New(inputType)
		if err := json.Unmarshal([]byte(input), ptr.Interface()); err != nil {
			return "", fmt.Errorf("unmarshal into %v: %w", inputType, err)
		}
		return f(extract(ptr.Elem()))
	}

This ensures:

• Primitive types (string, int, bool, float64 etc.) are unmarshaled correctly.
• Struct types continue to work as before.
• The inputType computed by ensureInputStructType[T]() is actually used during execution.

Tests

connect/tools

All TestNewTool_* tests were updated to include direct Execute() calls, verifying end-to-end unmarshaling behavior in addition to tool construction.

Fixed

• TestNewTool_PrimitiveWithMapPrimitive — corrected field name to scores.

134/134 tests passing